Conversation
Step 1 of #7052. - ansible/scripts/install-ansible.sh: bootstrap that installs OS prerequisites and ansible 10.x via pipx. Does not install galaxy collections or invoke any playbook. - ansible/playbooks/install_dev_env.yaml: tag-driven host installation playbook with no vars_prompt. Auto-derives rosdistro from Ubuntu version (22.04 -> humble, 24.04 -> jazzy). Signed-off-by: Mete Fatih Cırıt <mfc@autoware.org>
Step 1 of #7052. - Rename .github/workflows/setup-universe.yaml to health-check-ansible.yaml. - Replace setup-dev-env.sh invocation with the new install-ansible.sh + ansible-galaxy + install_dev_env playbook flow. - Drop ros_distro from the matrix; install_dev_env.yaml auto-derives it from the Ubuntu version. - Rename the job from setup-universe to install-dev-env. Branch protection that referenced "setup-universe / setup-universe" must be updated to "health-check-ansible / install-dev-env". Signed-off-by: Mete Fatih Cırıt <mfc@autoware.org>
xmfcx
requested review from
isamu-takagi,
mitsudome-r,
oguzkaganozt and
youtalk
as code owners
|
Thank you for contributing to the Autoware project! 🚧 If your pull request is in progress, switch it to draft mode. Please ensure:
|
Signed-off-by: Mete Fatih Cırıt <mfc@autoware.org>
Skip the workflow unless ansible/**, ansible-galaxy-requirements.yaml, or the workflow itself changed. Matches the pattern in health-check-pr.yaml. Signed-off-by: Mete Fatih Cırıt <mfc@autoware.org>
mitsudome-r
approved these changes
Apr 23, 2026
The cuda and tensorrt roles gate devel-package installation on 'install_devel == y' / 'install_devel == N'. universe.yaml previously supplied this via vars_prompt with default 'y'; install_dev_env.yaml must do the same explicitly since it has no prompts. Quoted to avoid PyYAML interpreting bare 'y' as boolean True; yamllint disable comment because quoted-strings rule does not account for truthy-ambiguous values. Signed-off-by: Mete Fatih Cırıt <mfc@autoware.org>
- ansible/roles/artifacts/defaults/main.yaml: default data_dir to
$HOME/autoware_data so the role no longer requires every caller to
supply it. (var-naming lint suppressed because the role tasks already
reference {{ data_dir }} unprefixed in 126 places; renaming would be
a separate cleanup.)
- ansible/playbooks/install_dev_env.yaml: reorder roles so optional
dev tooling and qt5ct_setup run before runtime deps; drop unused
tags (base from rmw, core from geographiclib).
Signed-off-by: Mete Fatih Cırıt <mfc@autoware.org>
Add actions/cache@v5 for /var/cache/apt/archives and /var/lib/apt/lists, keyed by the matrix image and a hash of ansible/** files. Disable docker-clean and set Keep-Downloaded-Packages so apt-get does not auto-delete cached debs. Speeds up repeat runs of the multi-GB CUDA/TensorRT installation steps. Signed-off-by: Mete Fatih Cırıt <mfc@autoware.org>
Reverts cda320f. The naive cache-on-PR pattern is a no-op across PRs (no main-side save means each PR starts cold), so it's pure CI overhead. The full strategy (push-to-main save trigger, single-instance enforcement, size-bounded discard, plus a separate workflow for artifact caching) is captured in .claude/todo.md and will land in a follow-up PR. Signed-off-by: Mete Fatih Cırıt <mfc@autoware.org>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Step 1 of Simplify ansible entrypoints: rename playbooks and drop setup-dev-env.sh #7052.
refactor(ansible): add install script and host dev env playbook
ansible/scripts/install-ansible.sh— bootstrap that installs OS prerequisites and ansible 10.x via pipx. Does not install Galaxy collections or invoke any playbook.ansible/playbooks/install_dev_env.yaml— tag-driven host installation playbook. Auto-derivesrosdistrofrom Ubuntu version (22.04 → humble, 24.04 → jazzy). Novars_prompt; selection via--tags/--skip-tags.ci(ansible): rename
setup-universeworkflow tohealth-check-ansible.github/workflows/setup-universe.yaml→health-check-ansible.yaml.setup-dev-env.shinvocation withinstall-ansible.sh+ansible-galaxy+install_dev_envplaybook.setup-universetoinstall-dev-env.ros_distroremoved (auto-derived from Ubuntu version).Existing playbooks and
setup-dev-env.share left in place; subsequent PRs in #7052 will rename and consolidate.Branch protection
If
setup-universe / setup-universewas a required status check, update protection rules tohealth-check-ansible / install-dev-envafter this lands.Test plan
run:health-checklabel applied; verifyhealth-check-ansible / install-dev-envsucceeds on bothubuntu:22.04andubuntu:24.04matrix legs.